-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
feat(prompts): support a visible status for un-hiding components #74228
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there a test we can add?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #74228 +/- ##
==========================================
- Coverage 79.94% 78.13% -1.82%
==========================================
Files 6836 6665 -171
Lines 307457 297923 -9534
Branches 51252 51281 +29
==========================================
- Hits 245808 232772 -13036
+ Misses 61225 58876 -2349
- Partials 424 6275 +5851
|
…74236) Depends on #74228 Relates to #74209 Adds a button to the "User Feedback" section of issue details page. Allows members to hide/unhide feedback from **all** issue details in a project. Persists this decision across reloads using the prompt backend Updated demo with chevron/drop-down style hiding: https://github.com/user-attachments/assets/43a02a63-98c0-42bd-8651-f1baeeb22218 Updated loading indicator: (you can see for a split sec at :12) https://github.com/user-attachments/assets/a2d57c51-e459-467b-9a9c-5e227050d9b7
Needed for #74209. We're using the prompt backend to persist a user's decision to hide/unhide user feedback in the issue details page. The goal is to be able to toggle between promptIsDismissed = true/false.